-
Notifications
You must be signed in to change notification settings - Fork 4
Enhance Langfuse Integration: Add /responses Support, Fix /threads Context, and Decouple Dependencies #248
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov ReportAttention: Patch coverage is 📢 Thoughts on this report? Let us know! |
Have you tested a case where you use threads and/or response endpoint for a user with no langfuse creds given, directly after you run these endpoint(s) for a user with langfuse creds given |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved.
Summary
Closes #210: Removes hard dependency on Langfuse for /threads and /response, allowing use with only OpenAI credentials.
Fixes #249: Eliminates race conditions caused by shared langfuse_context across async requests and background jobs.
Closes #244: Brings tracing to /responses, ensuring consistent Langfuse session grouping.
Key Changes:
Grouping of traces in Response API
To group traces by conversation:
Checklist
fastapi run --reload app/main.py
ordocker compose up
in the repository root and test.Notes
Please add here if any other information is required for the reviewer.